Skip to content

[webview_flutter_lwe] Remove Ecore API#1046

Open
JSUYA wants to merge 1 commit into
flutter-tizen:masterfrom
JSUYA:remove-ecore-webview-flutter-lwe
Open

[webview_flutter_lwe] Remove Ecore API#1046
JSUYA wants to merge 1 commit into
flutter-tizen:masterfrom
JSUYA:remove-ecore-webview-flutter-lwe

Conversation

@JSUYA

@JSUYA JSUYA commented Jun 19, 2026

Copy link
Copy Markdown
Member

Replace Ecore main-loop dispatch with g_idle_add_full in MessageDispatcher.

@JSUYA JSUYA self-assigned this Jun 19, 2026
@JSUYA JSUYA added the enhancement New feature or request label Jun 19, 2026
@JSUYA JSUYA force-pushed the remove-ecore-webview-flutter-lwe branch from c95d46b to 7e115e7 Compare June 19, 2026 05:25

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the webview_flutter_lwe package to version 0.4.1, replacing the Ecore API with GLib's g_idle_add_full for main thread task dispatching in message_dispatcher.cc. Feedback recommends using the GDestroyNotify callback of g_idle_add_full to safely deallocate the Param struct, ensuring memory is freed even if the idle source is removed before execution.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread packages/webview_flutter_lwe/tizen/src/message_dispatcher.cc Outdated
Replace Ecore main-loop dispatch with g_idle_add_full in MessageDispatcher.
@JSUYA JSUYA force-pushed the remove-ecore-webview-flutter-lwe branch from 7e115e7 to b14cb60 Compare June 19, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant